Skip to content

Ignore certain icons - #22

Merged
alexlanz merged 2 commits into
mainfrom
ignore-certain-icons
Aug 9, 2023
Merged

alexlanz merged 2 commits into
mainfrom
ignore-certain-icons

Conversation

@alexlanz

@alexlanz alexlanz commented Aug 9, 2023

Copy link
Copy Markdown
Member

No description provided.

@alexlanz
alexlanz requested a review from mmalfertheiner August 9, 2023 07:10
Comment thread generator.js Outdated
{ id: 'materialiconstwotone', postfix: 'TwoTone' },
]

const ignoredIcons = ['addchart']

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wdyt about adding a comment why we ignore this icon? Could be helpful in the future.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great idea, thanks.

@alexlanz
alexlanz merged commit 4ad077f into main Aug 9, 2023
@alexlanz
alexlanz deleted the ignore-certain-icons branch August 9, 2023 07:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants